Skip to content

Fix theme issues in Sass preprocessing - #2131

Merged
wedaly merged 1 commit into
masterfrom
db/fix-theme-name-issues
Jan 10, 2014
Merged

Fix theme issues in Sass preprocessing#2131
wedaly merged 1 commit into
masterfrom
db/fix-theme-name-issues

Conversation

@singingwolfboy

Copy link
Copy Markdown
Contributor

Sorry I didn't catch this earlier.

Reviewers: @wedaly @feanil

@wedaly

wedaly commented Jan 10, 2014

Copy link
Copy Markdown
Contributor

I'm still getting errors with this branch: https://gist.github.com/wedaly/8356552

I'm testing this in devstack using the command:

rake gather_assets[lms,devstack]

This runs using settings almost identical to the sandbox deploy.

@wedaly

wedaly commented Jan 10, 2014

Copy link
Copy Markdown
Contributor

with the last commit it's passing now :)

@wedaly

wedaly commented Jan 10, 2014

Copy link
Copy Markdown
Contributor

👍

@singingwolfboy

Copy link
Copy Markdown
Contributor Author

@wedaly Yep, I made the PR before I actually tested it locally. As soon as I found that error, I fixed it in the PR. You were too fast for me!

@feanil

feanil commented Jan 10, 2014

Copy link
Copy Markdown
Contributor

looks good.

wedaly pushed a commit that referenced this pull request Jan 10, 2014
Fix theme issues in Sass preprocessing
@wedaly
wedaly merged commit 4a3cffb into master Jan 10, 2014
@wedaly
wedaly deleted the db/fix-theme-name-issues branch January 10, 2014 16:03
xitij2000 pushed a commit to open-craft/openedx-platform that referenced this pull request Mar 17, 2021
…evelopment branch to development branch (openedx#2131)

picked the required commits from backup-ironwood-development branch to development
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants